Skip to content

feat(passport): Passport SDK marketing consent#2676

Merged
ImmutableJeffrey merged 5 commits into
mainfrom
ID-3893/passport-marketing-consent
Aug 8, 2025
Merged

feat(passport): Passport SDK marketing consent#2676
ImmutableJeffrey merged 5 commits into
mainfrom
ID-3893/passport-marketing-consent

Conversation

@CodeSchwert

@CodeSchwert CodeSchwert commented Aug 7, 2025

Copy link
Copy Markdown
Contributor

Hi👋, please ensure the PR title follows the below standards:

  • PR is titled with conventional commit style naming: type(scope): message. For example: feat(passport): my new feature
  • If you have introduced modification that necessitates immediate adjustments by this SDK's users to their applications, clients, or integrations to avert disruptions to existing features or functionalities, add a ! after the type(scope), for example feat(passport)!: my new breaking feature

Summary

ID-3893

  • Add marketing consent params to Passport login methods
  • Update the Passport SDK sample app to use the marketing consent param for direct login

Detail and impact of the change

Added

Changed

Deprecated

Removed

Fixed

Security

Anything else worth calling out?

The updated direct login methods on the SDK sample app all have the Unsubscribed value for testing purposes. Uses will be able to select whichever consent value when the call the the login method.

@CodeSchwert CodeSchwert requested a review from a team as a code owner August 7, 2025 08:53
@nx-cloud

nx-cloud Bot commented Aug 7, 2025

Copy link
Copy Markdown

View your CI Pipeline Execution ↗ for commit a6c0c85

Command Status Duration Result
nx affected -t build,lint,test ✅ Succeeded 3m 20s View ↗
nx run-many -p @imtbl/sdk,@imtbl/checkout-widge... ✅ Succeeded 1m 40s View ↗

☁️ Nx Cloud last updated this comment at 2025-08-08 04:21:33 UTC

@CodeSchwert CodeSchwert changed the title [ID-3893] Passport SDK marketing consent feat(passport): Passport SDK marketing consent Aug 7, 2025
@CodeSchwert CodeSchwert requested a review from a team as a code owner August 8, 2025 01:51
@ImmutableJeffrey ImmutableJeffrey added this pull request to the merge queue Aug 8, 2025
Merged via the queue into main with commit 262f265 Aug 8, 2025
8 checks passed
@ImmutableJeffrey ImmutableJeffrey deleted the ID-3893/passport-marketing-consent branch August 8, 2025 04:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants